src/loop/main.ts (1)

The createInterface import is added but not used in the runIterations function. It's only used in the runLoop function. It would be better to move this import to where it's actually used to improve modularity and reduce unnecessary imports in runIterations.

src/loop/main.ts (70)

The interactive variable checks for process.stdin.isTTY or process.env.TMUX. While process.stdin.isTTY correctly identifies an interactive terminal, relying on process.env.TMUX might not be universally reliable across all environments or tmux configurations. It might be more robust to check for a specific --interactive flag if interactive behavior is critical, or ensure that TMUX is a consistent indicator in your environment.

src/loop/pr.ts (4-17)

The buildDraftPrPrompt function now takes hasExistingPr as a parameter to conditionally build different prompts. This logic could be slightly cleaner by extracting the two prompt arrays into named constants or separate helper functions, improving readability and maintainability, especially if the prompts become more complex.

const PR_EXISTS_PROMPT_PARTS = [
  "A PR already exists for this branch. Send a follow-up commit to it.",
  (task: string) => `Task context:\n${task.trim()}`,
  "Commit your follow-up changes, push them, and return the commit in your final response.",
];

const NEW_PR_PROMPT_PARTS = [
  "Create a draft GitHub pull request for the current branch.",
  (task: string) => `Task context:\n${task.trim()}`,
  "Use `gh pr create --draft` with a clear title and description.",
  "If a PR already exists for this branch, do not create another one.",
  "Return the PR URL in your final response.",
];

const buildDraftPrPrompt = (task: string, hasExistingPr: boolean): string =>
  hasExistingPr
    ? PR_EXISTS_PROMPT_PARTS.map(part => typeof part === 'function' ? part(task) : part).join("\n\n")
    : NEW_PR_PROMPT_PARTS.map(part => typeof part === 'function' ? part(task) : part).join("\n\n");

src/loop/worktree.ts (1)

The rmSync import is added but it's only used within the moveFile function in defaultDeps. It might be cleaner to define moveFile directly within defaultDeps or ensure all file system operations are grouped logically.

src/loop/worktree.ts (109-111)

The moveFile implementation uses cpSync followed by rmSync. While this works, it's not an atomic operation. If the process crashes between cpSync and rmSync, the file might be duplicated or in an inconsistent state. For critical file movements, consider using fs.renameSync which is atomic, or implement more robust error handling and cleanup.

  moveFile: (source: string, target: string) => {
    renameSync(source, target);
  },

src/loop/worktree.ts (201)

The removal of the deps.pathExists(candidatePath) check here means that the tryCreateWorktree function will now attempt to create a worktree even if the path already exists. This might lead to isWorktreeConflict being triggered more often, which is handled, but it changes the initial check. Ensure this is the intended behavior.